Skip to content

Conversation

@vishwamartur
Copy link

@vishwamartur vishwamartur commented Jan 20, 2025

Related to #766

Add support for multiple monitors and provide screenshots of the visualization containing more than one monitor.

  • openadapt/utils.py

    • Update take_screenshot function to iterate through all monitors and capture screenshots.
    • Add a new function take_screenshots to return a list of screenshots from all monitors.
  • openadapt/visualize.py

    • Modify main function to handle and display screenshots from multiple monitors.
    • Update the HTML generation logic to include screenshots from all monitors.
  • openadapt/record.py

    • Update read_screen_events function to support capturing screenshots from multiple monitors.
  • tests/openadapt/test_utils.py

    • Add unit tests for the new take_screenshots function.

/claim #766

Related to OpenAdaptAI#766

Add support for multiple monitors and provide screenshots of the visualization containing more than one monitor.

* **openadapt/utils.py**
  - Update `take_screenshot` function to iterate through all monitors and capture screenshots.
  - Add a new function `take_screenshots` to return a list of screenshots from all monitors.

* **openadapt/visualize.py**
  - Modify `main` function to handle and display screenshots from multiple monitors.
  - Update the HTML generation logic to include screenshots from all monitors.

* **openadapt/record.py**
  - Update `read_screen_events` function to support capturing screenshots from multiple monitors.

* **tests/openadapt/test_utils.py**
  - Add unit tests for the new `take_screenshots` function.
@algora-pbc algora-pbc bot mentioned this pull request Jan 20, 2025
@algora-pbc
Copy link

algora-pbc bot commented Jan 20, 2025

💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe.

@abrichr
Copy link
Member

abrichr commented Jan 20, 2025

Thank you @vishwamartur! Can you please paste some screenshots of visualize.py and the dashboard? Also can you please upload the HTML file produced with visualize.py --cleanup=False?

@abrichr
Copy link
Member

abrichr commented Jan 17, 2026

Closing as part of the architecture transition. The legacy codebase has been moved to legacy/ and frozen at v0.46.0.

If this change is still needed, please:

  1. Open a PR against the relevant modular package repository
  2. Or re-open this PR targeting the legacy/ directory if it's a critical legacy fix

See PR #960 for details on the new meta-package architecture.

@abrichr abrichr closed this Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants